Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/async/package.json`:
- Around line 4-5: In the package.json file, update the placeholder metadata
fields: replace the "description" field value from "A template primitive
example." with an accurate description of what the async package does, and
replace the "author" field value from "Your Name <you@youremail.com>" with the
actual author's name and email address. These fields will be published to npm,
so ensure they contain real, meaningful information.
- Around line 35-47: The exports field in package.json declares a
`@solid-primitives/source` export condition that points to ./src/index.ts, but the
files array only includes the dist directory. This creates a mismatch where
consumers trying to use the `@solid-primitives/source` export will fail because
the src directory is not published to npm. Fix this by choosing one of two
approaches: either add "src" to the files array to publish source files
alongside dist, or remove the `@solid-primitives/source` export condition from the
exports field to prevent consumers from attempting to resolve unpublished source
files.

In `@packages/async/README.md`:
- Around line 13-18: The anchor fragments in the README table of contents use
camelCase format (fromStream, fromJSONStream, createAggregated, etc.) but
standard markdown slug generation converts headings to lowercase. Update all the
anchor fragment links in the primitive index to use lowercase versions
(fromstream, fromjsonstream, makeabortable, createabortable, makeretrying,
createaggregated) so they correctly resolve to the actual heading slugs in the
document.
- Around line 64-66: The README documentation contains mismatches with the
actual implementation for the fromJSONStream API. First, correct the function
label from "fromStream" to "fromJSONStream" in the definition block that shows
the actual function signature. Second, update all documentation examples and
option descriptions to use the correct option name "autoAbort" instead of
"noAutoAbort", ensuring the semantics are clearly documented that autoAbort
controls whether streams are automatically aborted (with inverse logic to what
the docs currently show). Apply these corrections to all instances mentioned
including the definition block around line 64-66 and the additional locations at
lines 108-109 and 118.

In `@packages/async/src/index.ts`:
- Around line 229-231: The toArray function uses a falsy check that incorrectly
converts valid falsy values like 0, false, and empty strings into empty arrays,
losing data during aggregation. Replace the ternary condition in toArray to
explicitly check for null or undefined values only (using == null or equivalent
checks) instead of relying on truthiness, so that valid falsy values like 0,
false, and "" are wrapped in arrays rather than discarded. Apply the same fix to
the other instance mentioned at lines 256-257.
- Around line 171-173: The isIterable and isAsyncIterable functions use
Object.hasOwn to check for Symbol.iterator and Symbol.asyncIterator, but this
only detects own properties and misses symbols on the prototype chain. Arrays,
strings, and generators have these symbols inherited from their prototypes,
causing makeRetrying to incorrectly treat them as single values. Replace
Object.hasOwn with the in operator for both isIterable and isAsyncIterable,
since the in operator checks the entire prototype chain and will correctly
identify all standard iterables and async iterables.
- Around line 195-197: The `retries` variable is being declared outside the
generator function and captured in the closure, causing state to leak and be
shared across multiple invocations. Additionally, using the `||` operator with
`options.retries` treats `0` as falsy, preventing `retries: 0` from being a
valid option to disable retries. Move the `retries` variable declaration inside
the generator function so each invocation gets its own independent retry
counter, and change the assignment from `options.retries || 3` to use the
nullish coalescing operator `??` instead to properly handle `0` as a legitimate
value.

In `@packages/async/stories/createAbortable.stories.tsx`:
- Around line 74-76: The input element with placeholder "type for autosuggest"
is using the onChange event handler which only fires on blur in Solid.js,
causing delayed autosuggest updates. Replace the onChange handler with onInput
on this input element to ensure the setQuery callback fires immediately on every
keystroke, providing instant suggestion responsiveness.

In `@packages/async/stories/fromJSONStream.stories.tsx`:
- Line 33: The JSON stream chunking in the anonymous function has an off-by-one
error in the slice end index calculation. The slice call currently subtracts 1
from the end index with `(idx + 1) * sliceLength - 1`, which removes one
character from each chunk and corrupts the JSON parsing. Remove the `- 1` from
the end index so that the slice correctly includes all characters up to the
proper boundary at `(idx + 1) * sliceLength`.

In `@packages/async/stories/fromStream.stories.tsx`:
- Line 33: The slice call in the callback function (around the `source.slice`
expression) is incorrectly subtracting 1 from the end index parameter, which
causes character loss at chunk boundaries because JavaScript's slice method
already treats the end index as exclusive. Remove the `- 1` from the end
parameter calculation in the slice call so it becomes `(idx + 1) * sliceLength`
instead of `(idx + 1) * sliceLength - 1`.

In `@packages/async/stories/makeAbortable.stories.tsx`:
- Around line 75-77: The input element in the makeAbortable.stories.tsx file is
using the onChange event handler, which in SolidJS only fires when the input
loses focus, causing a lag in the auto-suggest behavior. Replace the onChange
prop with onInput on the input element to ensure the setQuery callback fires on
every keystroke, providing real-time suggestions as the user types.

In `@packages/async/test/index.test.ts`:
- Line 168: Remove the debug console.log(result) statement from the retry test
in packages/async/test/index.test.ts. This console.log call serves no purpose in
the test and creates unnecessary noise in the CI/test output. Simply delete the
line containing console.log(result) from the test file.

In `@packages/resource/README.md`:
- Around line 11-12: The migration tip in the README.md file has unmatched
backtick markup that breaks markdown rendering. The text solid-js@>=2.0.0 has a
closing backtick but is missing the opening backtick, so it should be wrapped as
`solid-js@>=2.0.0` instead of solid-js@>=2.0.0` to properly render as inline
code.

---

Nitpick comments:
In `@packages/async/test/index.test.ts`:
- Around line 14-15: The helper streams in the test are not properly closing
when data is exhausted. In both locations where the code checks if done is true
and returns early (around the controller.enqueue calls), add a
controller.close() call before the return statement to ensure the stream
completion semantics are properly exercised. This applies to both instances
where this pattern appears in the file.